WHUPS! It looks like I jumped the gun for v83. Ah, well... sorry about that.

Codecov Report

❌ Patch coverage is 92.72727% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.04%. Comparing base (b7cf18f) to head (848cba8).
⚠️ Report is 6 commits behind head on master.

@@            Coverage Diff             @@
##           master    #4016      +/-   ##
==========================================
- Coverage   94.04%   94.04%   -0.01%     
==========================================
  Files         207      207              
  Lines       19174    19206      +32     
==========================================
+ Hits        18033    18063      +30     
- Misses        939      940       +1     
- Partials      202      203       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Could we retrigger the lint step? Something wrong with the GitHub API.

It looks like the test for ListByOrg could add a testBadOptions call to improve coverage.

It looks like the test for ListByOrg could add a testBadOptions call to improve coverage.

and possibly the other methods too... see #4019 for more details.

@gmlewis I fixed the test ListByOrg and it increases coverage slightly.

But it's impossible to cover these lines:

But it's impossible to cover these lines:

Even with something like this?

opts := &ListAllIssuesOptions{Filter: "\n"}

But it's impossible to cover these lines:

Even with something like this?

opts := &ListAllIssuesOptions{Filter: "\n"}

Yes. I tried the following test:

func TestIssuesService_ListAllIssues_badOptions(t *testing.T) {
	t.Parallel()
	ctx := t.Context()
	client, mux, _ := setup(t)

	mux.HandleFunc("/issues", func(w http.ResponseWriter, r *http.Request) {
		testMethod(t, r, "GET")
		testHeader(t, r, "Accept", mediaTypeReactionsPreview)
		fmt.Fprint(w, `[{"number":1}]`)
	})

	const methodName = "ListAllIssues"
	testBadOptions(t, methodName, func() (err error) {
		_, _, err = client.Issues.ListAllIssues(ctx, &ListAllIssuesOptions{
			Filter: "\n",
		})
		return err
	})
}

$ go test -run=^TestIssuesService_ListAllIssues_badOptions$ ./...
--- FAIL: TestIssuesService_ListAllIssues_badOptions (0.00s)
    issues_test.go:88: bad options ListAllIssues err = nil, want error
FAIL
FAIL    github.com/google/go-github/v82/github  0.571s
?       github.com/google/go-github/v82/test/fields     [no test files]
?       github.com/google/go-github/v82/test/integration        [no test files]
FAIL

Yes. I tried the following test:

Ah! You are absolutely right! The URL becomes "issues?filter=%0A". Sorry about that, @alexandear, and thank you for trying!

Thank you, @Not-Dhananjay-Mishra!
Merging.